Repository navigation
Conversation
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
codebase-memory-mcpthe primary graph MCP for Dev Plane while retaining GitNexus as a migration fallbackpackages/repo-intelrouting contractcodeintel-promotemigration gateArchitecture
Shared: codebase-memory + Zoekt canonical indexes.
Per worktree: native LSP,
ast-grep,rg/fd, compiler/tests.On demand: Joern. SCIP remains deferred until a measured cross-repository semantic gap justifies another index lifecycle.
packages/repo-intelexposes stable intents so callers do not depend on GitNexus/codebase-memory-specific tool names.Benchmark ground truth
The corpus is a curated set of required core production paths, not an exhaustive relevance set.
It currently covers:
Broad codebase-memory scenarios use deterministic BM25
search_graph(query=...); the Nulang caller scenario usestrace_path. Every codebase-memory call is explicitly scoped to the indexed project.Promotion rule
The migration gate is deliberately recall-first:
1.00on every scenariocodeintel-promoteexits non-zero for an unsafe result. GitNexus is not removed unless the live suite returnspromote: true.Reproducibility
The container benchmark worker pins Node 24, Go 1.26.8,
codebase-memory-mcp@0.11.0, andgitnexus@1.6.12, invokes the installed binaries directly, and isolates backend caches.Final
suite.jsonrecords:The normal portable lane is:
A pinned local live run is:
Woodpecker
.woodpecker/code-intelligence.yamlruns harness/corpus/provenance tests and repo-intel Go tests on the existinglinux/amd64pool..woodpecker/code-intelligence-bakeoff.yamlis manual-only. It needs Dev Plane activated in Woodpecker plus repository secretcodeintel_github_tokenwith read-only Contents access to the private Adacavo and Nulang Cloud repos. The workflow removes/unsets the temporary Git credential before indexing/querying.See
docs/code-intelligence-bakeoff-runbook.mdfor the complete operation/migration sequence.Verification status
TDD covers routing, scorer behavior, corpus contracts, provenance binding, promotion policy, and CLI exit behavior.
The chat execution sandbox cannot run the authoritative final suite: it lacks the required Go 1.25.11+ toolchain/backends/canonical private clones and cannot download them because outbound networking is restricted. The repository-side portable and Woodpecker lanes are therefore the authoritative execution paths for the final branch state.
GitHub-hosted Actions remains independently broken at admission level. Main-branch run
36910222862already showed the same failure mode before this PR:Lint,Build, andTestwere created but executed zero steps. Later PR runs show the same zero-step/no-log behavior. Do not interpret those runs as compile/test failures.Migration sequence
Do not remove Adacavo's GitNexus-specific safety/freshness assets in this PR.
After a trusted worker produces
promote: true:Nulang and Nulang Cloud already point at codebase-memory and do not need a GitNexus-removal step from this migration branch.